Skip to content

allow users to create threads for their reports #484

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 4 commits into from
Jun 24, 2024
Merged

allow users to create threads for their reports #484

merged 4 commits into from
Jun 24, 2024

Conversation

@danthe1st
Copy link
Member

@danthe1st danthe1st commented Mar 4, 2024

When reporting a user, this PR adds a button which automatically creates a private thread where the reporter can provide additional information.

image
image
image
image

Deploy notes

The config moderationConfig.reportUserThreadHolderId needs to be set when this is used.

@danthe1st danthe1st requested a review from a team as a code owner March 4, 2024 10:06
@jasonlessenich
Copy link
Member

jasonlessenich commented Mar 31, 2024

bump

@liquiddevelopmentnet
Copy link
Member

liquiddevelopmentnet commented Mar 31, 2024

Which channel in the server do you have in mind for being the thread parent?

@danthe1st
Copy link
Member Author

danthe1st commented Mar 31, 2024 •
edited
Loading

Which channel in the server do you have in mind for being the thread parent?

I thought about using the #general channel.

@jasonlessenich
Copy link
Member

jasonlessenich commented Apr 1, 2024

Which channel in the server do you have in mind for being the thread parent?

I thought about using the #general channel.

Wouldn't a forum channel in the staff category be more suitable? Idk if you can restrict a user to only seeing this single thread tho

@danthe1st
Copy link
Member Author

danthe1st commented Apr 1, 2024

Which channel in the server do you have in mind for being the thread parent?

I thought about using the #general channel.

Wouldn't a forum channel in the staff category be more suitable? Idk if you can restrict a user to only seeing this single thread tho

This is not possible AFAIK.
Sending messages in threads requires being able to access thread thread and forum channels provide no way of restricting threads.

Furthermore, I don't think users should be able to view some report channel without ever using that feature.

@jasonlessenich
Copy link
Member

jasonlessenich commented Apr 1, 2024

Which channel in the server do you have in mind for being the thread parent?

I thought about using the #general channel.

Wouldn't a forum channel in the staff category be more suitable? Idk if you can restrict a user to only seeing this single thread tho

This is not possible AFAIK.
Sending messages in threads requires being able to access thread thread and forum channels provide no way of restricting threads.

Furthermore, I don't think users should be able to view some report channel without ever using that feature.

Well, they could only have permission to see this channel if they have an open thread
But yeah, ig #general is fine

@MoonTM-GIT MoonTM-GIT merged commit 3c9a497 into Java-Discord:main Jun 24, 2024
@danthe1st danthe1st deleted the report-threads branch June 25, 2024 08:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jasonlessenich jasonlessenich jasonlessenich approved these changes

@liquiddevelopmentnet liquiddevelopmentnet liquiddevelopmentnet approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@danthe1st @jasonlessenich @liquiddevelopmentnet @MoonTM-GIT